Operation has No Effect (ONE)

Description:

ONE detects arithmetic expressions that always return one of their operands' value. This message is produced only for expressions that have at least one operand which is not a compile-time constant. The examined operations are described below.

• The addition (+) operation has no effect when one of the addends is zero (0).
• The subtraction (-) operation has no effect when the subtrahend is zero (0).
• The multiplication (*) operation has no effect when either the multiplicand or the multiplier is one (1).
• The division (/) operation has no effect when the divisor is one (1).
• The remainder (%) operation has no effect when the right operand is either greater or less than zero, and the absolute value of the left operand is less than the absolute value of the right operand. In this case, x % y == x or x % y == -x.
• A field or local variable is assigned to itself.

Incorrect:

class MenuItem {
    private String name;
    private int index;
	
    Item(String n, int index) {
        this.name = name;
        this.index = index;
    }
	
    int getPosition() {
        int bases = 0;
        return index + bases;
    }
}

Correct:

class MenuItem {
    private String name;
    private int index;
	
    Item(String name, int index) {
        this.name = name;
        this.index = index;
    }
	
    int getPosition() {
        return index;
    }
}